sec(gitleaks): commit-independent allowlists for permanent fixtures (backend#1404) — baseline retired - #457
Merged
Merged
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
Author
|
FR on staging → Ready for prod (staging FR sweep, 2026-08-06). Basis: CI/tooling-only change with no product runtime surface — verified green through code review + Bugbot on develop and the fr-gated staging promotion. No product-behavior verification required for this class. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Three-part change:
.gitleaks.tomlwith one commit-independent allowlist block: synthetic idempotency key in submit tests — 1 baseline entry (the baseline's only entry)..gitleaks-baseline.json— its single entry is the test fixture the toml now covers.gitleaks-baseline: .gitleaks-baseline.jsoninput line from.github/workflows/code-quality-caller.yml(and only that line).Why retire the baseline: its only entry was pinned to commit + file + line, so a future reformat of that line would have un-suppressed it — the exact defect backend#1404 names. The path+regex allowlist covers the fixture commit-independently, so it survives reformatting.
gitleaks 8.30.1 auto-loads the repo-root
.gitleaks.toml, so no workflow change is needed to activate it — the caller edit here only removes the now-redundant baseline input.[extend] useDefault = trueis present, so the default ruleset stays fully armed.Part of tracebloc/backend#1404
🤖 Generated with Claude Code
Note
Low Risk
CI secret-scan configuration only; no application runtime, auth, or data-path changes.
Overview
Gitleaks no longer uses a baseline file for the single known false positive in submit tests. A new
.gitleaks.tomlallowlists the syntheticIdempotencyKeyfixture in*_test.govia path + line regex, so suppressions survive line moves and reformats instead of being tied to commit/file/line..gitleaks-baseline.jsonis removed, andcode-quality-caller.ymldrops thegitleaks-baselineinput only; gitleaks still loads the default ruleset via[extend] useDefault = true.Reviewed by Cursor Bugbot for commit a60cce1. Bugbot is set up for automated code reviews on this repo. Configure here.